Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

re-enable flaky test #14032

Merged
merged 1 commit into from
Feb 4, 2019
Merged

Conversation

apeforest
Copy link
Contributor

Description

According to #10973, this test should no longer be flaky. Re-enable it to test deconvolution operators

@apeforest
Copy link
Contributor Author

@mxnet-label-bot add [pr-awaiting-review]

@marcoabreu marcoabreu added the pr-awaiting-review PR is waiting for code review label Jan 31, 2019
@anirudh2290
Copy link
Member

Did you run this test repeatedly ?

@apeforest
Copy link
Contributor Author

@anirudh2290 I ran 10K times and all passed. I think this test is good to go.

@apeforest
Copy link
Contributor Author

@mxnet-label-bot update [pr-awaiting-merge]

@marcoabreu marcoabreu added pr-awaiting-merge Review and CI is complete. Ready to Merge and removed pr-awaiting-review PR is waiting for code review labels Feb 2, 2019
@anirudh2290 anirudh2290 merged commit 3d809c1 into apache:master Feb 4, 2019
stephenrawls pushed a commit to stephenrawls/incubator-mxnet that referenced this pull request Feb 16, 2019
vdantu pushed a commit to vdantu/incubator-mxnet that referenced this pull request Mar 31, 2019
haohuanw pushed a commit to haohuanw/incubator-mxnet that referenced this pull request Jun 23, 2019
@apeforest apeforest deleted the test/enable_flaky_test branch August 23, 2019 17:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr-awaiting-merge Review and CI is complete. Ready to Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants